Skip to content

removed P100s from pricing table#1208

Merged
janamakar merged 1 commit intomainfrom
janamakar-patch-1
Mar 25, 2026
Merged

removed P100s from pricing table#1208
janamakar merged 1 commit intomainfrom
janamakar-patch-1

Conversation

@janamakar
Copy link
Copy Markdown
Collaborator

No description provided.

Signed-off-by: Jana Makar <36392208+janamakar@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 25, 2026

Test deployment successful!! (2026-03-25T21:16:50Z)
Preview available at https://callumwalley.github.io/mkdocs-demo-deploy/nesi/support-docs/janamakar-patch-1

Seems the following pages differ;



See all deployed demo sites

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

Ahoy there! This here pull request has bravely sent the P100 GPU pricing to the briny deep, removing it from the pricing table. However, it seems ye've left a few ghostly traces of the P100 scattered about the other charts and maps. A truly diligent seadog would ensure all mentions of the P100 be scrubbed clean from docs/Software/Containers/NVIDIA_GPU_Containers.md and docs/Batch_Computing/Using_GPUs.md, lest yer crew of users be led astray by outdated examples and scripts.

| ***High Performance Computing (HPC) and Data Analytics*** | | |
| Mahuika Milan HPC core hours (per CPU core hour) | $0.08 | $0.09 |
| Mahuika Genoa HPC core hours (per CPU core hour) | $0.08 | $0.09 |
| P100 Graphical Processing Unit (GPU) hours (per GPU hour) | $0.35 | $0.40 |
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

So, ye've decided to send the P100 pricing to Davy Jones' locker. A bold move. But it seems ye've been a bit hasty and left the ghost of the P100 hauntin' other parts of yer documentation. A truly thorough captain would ensure all traces are gone.

I've spied mentions of the P100 still lingerin' in these files:

  • docs/Software/Containers/NVIDIA_GPU_Containers.md has an example script that still calls for a P100. A poor soul might copy that and find their job fails.
  • docs/Batch_Computing/Using_GPUs.md shows an example output with a P100. Confusin', it is.

While docs/Getting_Started/FAQs/Mahuika_HPC3_Differences.md correctly states they're retired, these other mentions will surely lead some landlubbers astray. Best be tidyin' that up, lest ye want a mutiny on yer hands from confused users.

@janamakar janamakar merged commit c7c5a81 into main Mar 25, 2026
8 checks passed
@janamakar janamakar deleted the janamakar-patch-1 branch March 25, 2026 21:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant